Skip to content

feat: support RELAYPLANE_PROXY_PORT and RELAYPLANE_PROXY_HOST env vars#3

Open
jtalborough wants to merge 1 commit intoRelayPlane:mainfrom
jtalborough:feat/cli-env-vars
Open

feat: support RELAYPLANE_PROXY_PORT and RELAYPLANE_PROXY_HOST env vars#3
jtalborough wants to merge 1 commit intoRelayPlane:mainfrom
jtalborough:feat/cli-env-vars

Conversation

@jtalborough
Copy link

Summary

  • Adds environment variable support for RELAYPLANE_PROXY_PORT and RELAYPLANE_PROXY_HOST in src/cli.ts
  • CLI flags (--port, --host) still override env vars
  • Default values unchanged (port 4100, host 127.0.0.1)
  • Enables container deployments to configure binding without CLI flags

Changes

  • src/cli.ts: Read RELAYPLANE_PROXY_PORT and RELAYPLANE_PROXY_HOST env vars as defaults

Test plan

  • npm run build compiles cleanly
  • RELAYPLANE_PROXY_PORT=4801 RELAYPLANE_PROXY_HOST=0.0.0.0 npm start binds to correct port/host
  • --port and --host CLI flags still override env vars
  • Without env vars, defaults remain 4100/127.0.0.1

Allow port and host to be configured via environment variables, with CLI
flags still taking precedence. This enables container deployments to set
defaults without modifying the command line.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant